New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add SVG Sprite Module for icons #1808
Conversation
Size Change: -1.89 kB (0%) Total Size: 851 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I have a couple clarifying questions but in the meantime the Storybook fix is the only requested change.
Full-stack documentation: https://docs.openverse.org/_preview/1808 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
21225a5
to
83d4b5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM once the playwright tests pass 🙂. The plan to remove the other svg module in a separate PR sounds good. Is there an issue to do that?
9d185d8
to
05969d8
Compare
I'll open the issue once the Playwright tests pass :) |
05969d8
to
e727a7d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! The added documentation is very helpful too.
Fixes
Fixes #985 by @obulat
Description
This PR adds the Nuxt SVG Sprite Module for icons.
To minimize changes, I wrapped the
SvgIcon
component provided by the module in theVIcon
component.This PR also adds documentation page about the icons. I'm not sure where it should be placed.
Testing Instructions
All the snapshot tests should pass.
Checklist
Update index.md
).main
) ora parent feature branch.
errors.
Developer Certificate of Origin
Developer Certificate of Origin